2.3-code review
实际上,Code Review可以为我们带来大量的好处:
- 提高代码质量:Code Review可以帮助开发者发现潜在的问题、漏洞和错误,以及提供更好的实现方式,从而提高代码质量。
- 减少维护成本:在Code Review过程中,可以发现代码中的问题和漏洞,并及时修改,避免了在上线后出现问题需要大量时间和精力进行维护的情况。
- 加强团队协作:Code Review可以促进团队内成员之间的交流和协作,提高团队整体的代码质量和开发效率。
- 提高代码可读性:Code Review可以帮助开发者发现代码可读性的问题,例如命名、注释、代码结构等方面,从而提高代码的可读性。
- 检查代码安全性:Code Review可以检查代码中的安全问题,例如XSS、CSRF、SQL注入等,从而提高代码的安全性。
意义
code review 更像是将统一的编程思想推广到整个团队,让所有人的思想保持一致
一开始 leader 一人发起,然后就可以交给团队内部进行自我 review,来了新人,也能保证代码团队内部一致
常见需要Code Review的问题点:
- 重要模块整体review
- 滥用全局变量
- 命名规范
- 闭包内部变量未被销毁
- 计时器是否及时清理
- 监听事件是否有解绑
- 第三方库的销毁函数,在页面卸载时也需要调用,比如EventBus:
- v-if 指令导致的内存泄露
- 异步操作是否有异常处理
- 组件是否需要拆分
- TypeScript中滥用any
- 未用防抖节流
- 动画减少回流与重绘
- 关键业务未加注释
- 避免重复注释
- 注释业务上的作用,而不是翻译一下英文名称